Skip to content

Enable auto-merge for CodeSnip organization members - #49

Merged
MstfSlm38 merged 1 commit into
26Q2.5from
automation/org-member-automerge
Nov 4, 2026
Merged

MstfSlm38 merged 1 commit into
26Q2.5from
automation/org-member-automerge

Conversation

@MstfSlm38

Copy link
Copy Markdown
Collaborator

Summary

  • Automatically enable GitHub auto-merge for pull requests opened by CodeSnip organization owners and members.
  • Limit the automation to pull requests targeting 26Q2.5.
  • Leave pull requests from outside the organization untouched.

How it works

The workflow uses pull_request_target to read the pull request's GitHub-provided author_association, then runs gh pr merge --auto --merge without checking out or executing code from the pull request. GitHub waits for required checks and branch protection requirements before merging.

Repository setting required

Allow auto-merge must be enabled in Settings → General → Pull Requests. The repository must also permit merge commits for the selected --merge method.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-04T14:35:09.608673Z e8fac6d PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

💡 Codex Review


P2 Badge Remove or satisfy the missing test gate

For every created release, this step exits with Missing script: "test" because package.json defines no test script. Since publish-npm depends on build, npm publish never runs. Add a test script or remove this gate.


JSON.parse(rawData);

P2 Badge Validate the imported backup shape

If a user selects valid JSON that is not a CodeSnip snippet array, such as {}, this parse-only check accepts it and the import flow overwrites all_snippets_data with it. On reload, the snippet renderer calls .forEach on that object and throws, making the saved snippets inaccessible. Validate the backup structure before returning success.


pageTitle.innerText = titleMap[category][currentLang];

P2 Badge Provide titles for Spanish category filters

When the app language is Spanish, clicking All or Favorites reaches this map with currentLang === 'es', but neither title entry has an es value. The assignment therefore changes the page title to undefined; add Spanish entries or fall back to an available translation.


const code = document.createElement("code"); code.id = String(snip.id); code.style.fontSize = fontSize; code.textContent = source; pre.appendChild(code);

P2 Badge Preserve Prism language classes when rendering snippets

q3-5.js replaces the renderer with safeRenderSnips, but this code element gets no language-* class. Prism's subsequent highlightAllUnder call therefore does not select it, so snippets lose syntax highlighting. Restore the category-based language class while keeping the code text safely assigned.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@MstfSlm38
MstfSlm38 merged commit ddb04c8 into 26Q2.5 Nov 4, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant